Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(ecmascript)!: switch to ES2024 #2505

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

chore(ecmascript)!: switch to ES2024 #2505

wants to merge 2 commits into from

Conversation

ferferga
Copy link
Member

The ECMAScript upgrade is needed for Object.groupBy, which we use across the codebase

This can't be merged until TypeScript 5.7 has ubiquituous support by VSCode, ESLint and vue-tsc

@jellyfin-bot
Copy link

jellyfin-bot commented Nov 25, 2024

Cloudflare Pages deployment

Latest commit b1271d5
Status ❌ Failure. Check workflow logs for details
Preview URL ❓ Not available
Preview alias ❓ Not available
Type 🔀 Preview

View build logs

@ferferga
Copy link
Member Author

Something I've been forgetting to update is the browserslist query. However, there's no possibility to query by ECMAScript version: browserslist/browserslist#795

TODO: Investigate with CanIUse or browserslist themselves in a way to support querying for specific ecma versions, since it would be much easier and maintenable to do it that way.

@ferferga ferferga changed the title chore(ecmascript): switch to ES2024 chore(ecmascript)!: switch to ES2024 Dec 28, 2024
@ferferga ferferga force-pushed the es2024 branch 2 times, most recently from 23221f8 to 7324af4 Compare December 29, 2024 02:32
@jellyfin-bot jellyfin-bot added the merge conflict Something has merge conflicts label Jan 24, 2025
@jellyfin-bot jellyfin-bot removed the merge conflict Something has merge conflicts label Jan 24, 2025
For some reason, the client throwed an error about incorrect attribute
when using Docker buildx's docker-container driver.
The build worked flawlessly when using the normal docker build command
(which is now an alias to docker buildx build, but with the docker driver)
and in the host system using normal commands, and that's why it went unnoticed.

Really strange error... Perhaps something encoding related in
the outputted files?

Either way, this commit fixes it by removing the trailing double quotes
from the component!
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In progress
Development

Successfully merging this pull request may close these issues.

2 participants